Skip to content

refactor: comprehensive repository cleanup and organization#21

Open
superninja-app[bot] wants to merge 4 commits intomainfrom
fix/repository-cleanup
Open

refactor: comprehensive repository cleanup and organization#21
superninja-app[bot] wants to merge 4 commits intomainfrom
fix/repository-cleanup

Conversation

@superninja-app
Copy link
Copy Markdown

@superninja-app superninja-app bot commented Mar 9, 2026

Summary

Comprehensive repository cleanup addressing 50+ identified issues across structure, versioning, security, and organization.

Changes

Phase 1: Cleanup

  • Removed 300+ sandbox artifact files (outputs/, summarized_conversations/, build logs)
  • Removed duplicate directories (V-Sentinel/, V-Sentinel-IOCs/, V-Sentinel-SDKs/)
  • Moved session reports to archive/
  • Fixed .gitignore (track Cargo.lock, exclude sandbox artifacts)
  • Fixed script permissions (chmod +x)

Phase 2: Version Unification

  • Fixed 15 modules to use version.workspace = true
  • Updated workspace version from 2.1.0 → 2.1.2
  • Fixed gateway dependencies (tower-http compression-full, axum features)
  • Documented broken modules as TODO in workspace Cargo.toml

Phase 3: GitHub Security & Templates

  • Created SECURITY.md with vulnerability reporting policy
  • Created .github/CODEOWNERS for code ownership
  • Created .github/dependabot.yml for automated dependency updates
  • Created issue templates (bug_report.yml, feature_request.yml)
  • Created pull_request_template.md

Stats

  • 413 files changed (+3,669 / -22,987 lines)
  • Majority of deletions are sandbox artifacts that should never have been committed

Known Remaining Work

  • Modules excluded from workspace due to compilation errors: src/network, src/security, src/tools, src/services/*
  • These need individual fix branches to resolve 80+ compilation errors before workspace inclusion
  • Failing CI tests need separate investigation

Phase 1 - Cleanup & Hygiene:
- Remove sandbox artifacts (outputs/, summarized_conversations/)
- Remove empty V-Sentinel/ submodule reference
- Remove duplicate directories (V-Sentinel-IOCs/, V-Sentinel-SDKs/)
- Remove build artifacts (build_check.log, cloc_output.json)
- Move session reports to archive/
- Fix .gitignore (remove Cargo.lock exclusion, add sandbox artifacts)
- Fix script permissions (chmod +x)
- Add Cargo.lock to version control (binary crate)

Phase 2 - Versioning & Workspace:
- Unify 15 modules to use version.workspace = true
- Update workspace version to 2.1.2
- Fix gateway dependencies (tower-http compression, axum headers)

Phase 3 - GitHub Security & Templates:
- Add SECURITY.md with vulnerability reporting policy
- Add .github/CODEOWNERS for code review assignments
- Add .github/dependabot.yml for automated dependency updates
- Add issue templates (bug report, feature request)
- Add pull request template
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant